Skip to content

Nan checks for ncr and fac#82

Open
juntuu wants to merge 2 commits intocodeplea:masterfrom
juntuu:nan-check
Open

Nan checks for ncr and fac#82
juntuu wants to merge 2 commits intocodeplea:masterfrom
juntuu:nan-check

Conversation

@juntuu
Copy link

@juntuu juntuu commented Jul 26, 2021

Added nan check for ncr and fac functions.
Now both will return nan, when given nan as parameter.

This should resolve #62

Also added .gitignore with the artifacts from make

juntuu added 2 commits July 26, 2021 11:20
Now both `fac` and `ncr` will return NAN, if any argument is NAN.
Ignore object files and executables from make targets.

Also added repl-readline to make clean target.
@invd
Copy link

invd commented Jun 18, 2024

I can confirm that the better NAN handling resolves the slow ncr() behavior 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nested ncr() calls can be very slow (potential denial of service)

2 participants